]> dgit.raspbian.org Git - gnome-shell.git/commit
gdm: don't show fingerprint message right away
authorRay Strode <rstrode@redhat.com>
Tue, 11 Oct 2011 17:12:18 +0000 (13:12 -0400)
committerRay Strode <rstrode@redhat.com>
Tue, 11 Oct 2011 21:25:44 +0000 (17:25 -0400)
commit8529ca70af24b5b3863cbef52193cfe3d4f70c3f
treec811f0f59f8486d459df17568de8bf96dbcc5dc6
parent1a8d78212fcdcda185c93d4fe45324b560da9de3
gdm: don't show fingerprint message right away

Right now we show "(or swipe finger)" at the user login prompt
any time we detect a fingerprint reader.

Checking for the presense of a fingerprint reader isn't really
sufficient for knowing if it is appropriate or not to show the
message, though. Often, a user's fingerprint won't be enrolled
in the system even if the machine has a fingerprint reader.

In this scenario, we end up in a situation where the code will
fade out the message right after fading it in, or worse, fade
out the message while fading it in.

The former case looks flickery and bad, and the latter case
causes the login dialog to lock up since it never completes its
"show prompt" animation and we don't procede with the login
process until after that animation.

If a user is enrolled in the system, the fingerprint pam module
tries to tell the user to swipe their finger.  We never show the
user that message because it's redundant with our own "(or swipe
finger)" message and because it uses techy words like "UPEK" and
"TouchStrip".

This commit changes the code to defer showing "(or swipe
finger)" until the fingerprint pam module forwards us its own
message. This makes it less likely we'll show the message when
fingerprint login won't work, and also removes the fingerprint
animation from the critical path "show prompt" animation.

https://bugzilla.gnome.org/show_bug.cgi?id=660492
js/gdm/loginDialog.js